Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issue #12 branches with / in it are cut-off #13

Merged
merged 1 commit into from
May 15, 2015

Conversation

dgollub
Copy link
Contributor

@dgollub dgollub commented May 8, 2015

This fixes the cut off branch names when a branch has a / in the name.
Example: "dev/2.7.x" was cut off to "2.7.x".

Let me know what you think.

I left the original logic intact in case there are only two / in the ref: line, but otherwise it will chomp off the whole ref: refs/heads/ part and return everything after that. Not sure if this is the ideal solution, but it seems to work in this case.

PS: I installed this tool on our servers and my coworkers love it. Thanks again.

This fixes the cut off branch names when a branch has a / in the name.
Example: "dev/2.7.x" was cut off to "2.7.x".
@mrkline
Copy link
Owner

mrkline commented May 14, 2015

LGTM - thanks for the help! I was on vacation for the past two weeks so I didn't see this until now. I'll merge it tomorrow.

mrkline added a commit that referenced this pull request May 15, 2015
Fix issue #12 branches with / in it are cut-off
@mrkline mrkline merged commit 49506b4 into mrkline:master May 15, 2015
@dgollub dgollub deleted the fix-issue-12 branch May 18, 2015 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants